Skip to content

Add validated details - #707

Merged
adamlam2-amd merged 18 commits into
mainfrom
add_validated_details
Aug 18, 2026
Merged

Add validated details#707
adamlam2-amd merged 18 commits into
mainfrom
add_validated_details

Conversation

@lucbruni-amd

@lucbruni-amd lucbruni-amd commented Aug 13, 2026

Copy link
Copy Markdown
Collaborator

Opened on behalf of Adam (@adamlam2-amd).

This PR adds:

  • memory requirement for each playbook. Each playbook will have a static minimum memory requirement based on the models used.
  • validation details based on manual/automated testing per playbook per device and OS. Details on how that resolves is as follows:

Validated versions — how they resolve

  • Built at request time from 3 sources: dependency-versions.json (central version per dep, keyed by device category + OS), registry.json (versionable flag + display name), and each playbook.json's validatedVersions (per-playbook overrides).
  • Per device+OS: take the playbook's @require deps (scoped to the matching @os block, versionable only) → look up versions in the central file → apply validatedVersions on top (same-name key overrides, new keys add).
  • Net effect: each playbook lists only the software it uses (e.g. Lemonade playbook shows Lemonade, not ROCm). Shared version bumps = one line in the central file.
  • Apps show a pinned version on HaloBox and "Latest" on APU/dGPU (via a default fallback); "" hides a row.
    Implemented in route.ts (buildValidation); returns resolved validation[device][os].
  • Note: override keys must match the registry display name exactly, or they duplicate instead of override.

@adamlam2-amd

Copy link
Copy Markdown
Collaborator

@lucbruni-amd @danielholanda
This PR doesn't change anything fundamental to the tests or functionality of the playbooks. Rather, it adds metadata in the form of memory requirements, primary model, and validation details to the playbooks.

Going to merge and we can re-check if any issues with the staging website.

@adamlam2-amd
adamlam2-amd merged commit 4996176 into main Aug 18, 2026
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants